-
Notifications
You must be signed in to change notification settings - Fork 113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create OCM share from sciencemesh service #3695
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@michielbdejong now /ocm/send
is /sciencemesh/create-share
!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Maybe you can double-check that this endpoint returns a 401 when you hit it without any authentication.
FYI I merged this branch into https://github.com/cs3org/reva/tree/sciencemesh-dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As there were no objections, let's go ahead and fix the authentication as discussed, and then we merge this.
Co-authored-by: Giuseppe Lo Presti <giuseppe.lopresti@cern.ch>
Co-authored-by: Giuseppe Lo Presti <giuseppe.lopresti@cern.ch>
Co-authored-by: Giuseppe Lo Presti <giuseppe.lopresti@cern.ch>
The PR is now ready |
* create share from sciencemesh service * make /create-share unprotected * add changelog * fix linter! * Update internal/http/services/sciencemesh/share.go Co-authored-by: Giuseppe Lo Presti <giuseppe.lopresti@cern.ch> * Update internal/http/services/sciencemesh/share.go Co-authored-by: Giuseppe Lo Presti <giuseppe.lopresti@cern.ch> * Update internal/http/services/sciencemesh/sciencemesh.go Co-authored-by: Giuseppe Lo Presti <giuseppe.lopresti@cern.ch> * removed old code * fix import names --------- Co-authored-by: Giuseppe Lo Presti <giuseppe.lopresti@cern.ch>
* create share from sciencemesh service * make /create-share unprotected * add changelog * fix linter! * Update internal/http/services/sciencemesh/share.go Co-authored-by: Giuseppe Lo Presti <giuseppe.lopresti@cern.ch> * Update internal/http/services/sciencemesh/share.go Co-authored-by: Giuseppe Lo Presti <giuseppe.lopresti@cern.ch> * Update internal/http/services/sciencemesh/sciencemesh.go Co-authored-by: Giuseppe Lo Presti <giuseppe.lopresti@cern.ch> * removed old code * fix import names --------- Co-authored-by: Giuseppe Lo Presti <giuseppe.lopresti@cern.ch>
As in the recent refactor of the OCM service the endpoint /ocm/share was removed, this PR re-add it back.
Closes pondersource/sciencemesh-php#166